Skip to content

add support for textDocument/definition#50

Merged
aeschli merged 3 commits intomicrosoft:masterfrom
ananthakumaran:definition
Apr 27, 2020
Merged

add support for textDocument/definition#50
aeschli merged 3 commits intomicrosoft:masterfrom
ananthakumaran:definition

Conversation

@ananthakumaran
Copy link
Copy Markdown
Contributor

JSON Schema, Swagger, etc use $ref to reference reusable json snippets. Add the ability to jump to definition from $ref.

ezgif com-video-to-gif

JSON Schema, Swagger, etc use $ref to reference reusable json
snippets. Add the ability to jump to definition from $ref.
@msftclas
Copy link
Copy Markdown

msftclas commented Mar 7, 2020

CLA assistant check
All CLA requirements met.

Copy link
Copy Markdown
Collaborator

@aeschli aeschli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool stuff.
Maybe its better forfindDefinition to return a Promise so that we, in the future also can support external links (url#ref)

@ananthakumaran
Copy link
Copy Markdown
Contributor Author

updated

@aeschli aeschli merged commit 1fadf89 into microsoft:master Apr 27, 2020
@aeschli aeschli added this to the April 2020 milestone Apr 27, 2020
@aeschli
Copy link
Copy Markdown
Collaborator

aeschli commented Apr 27, 2020

Thanks @ananthakumaran !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants